- 
                Notifications
    You must be signed in to change notification settings 
- Fork 15
Remove reference to the iostack from info fetcher #1287
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove reference to the iostack from info fetcher #1287
Conversation
| ✅ Tests 🎉 All green!❄️ No new flaky tests detected 🔗 Commit SHA: 83d1cf3 | Docs | Datadog PR Page | Was this helpful? Give us feedback! | 
| BenchmarksComparisonBenchmark execution time: 2025-10-29 21:40:39 Comparing candidate commit 83d1cf3 in PR branch  Found 4 performance improvements and 1 performance regressions! Performance is the same for 50 metrics, 2 unstable metrics. scenario:benching deserializing traces from msgpack to their internal representation
 scenario:credit_card/is_card_number/37828224631000521389798
 scenario:credit_card/is_card_number_no_luhn/37828224631000521389798
 CandidateCandidate benchmark detailsGroup 1
 
 
 Group 2
 
 
 Group 3
 
 
 Group 4
 
 
 Group 5
 
 
 Group 6
 
 
 Group 7
 
 
 Group 8
 
 
 Group 9
 
 
 Group 10
 
 
 Group 11
 
 
 Group 12
 
 
 Group 13
 
 
 Group 14
 
 
 Group 15
 
 
 Group 16
 
 
 Group 17
 
 
 BaselineOmitted due to size. | 
| Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@            Coverage Diff             @@
##             main    #1287      +/-   ##
==========================================
+ Coverage   71.91%   72.09%   +0.18%     
==========================================
  Files         368      368              
  Lines       58146    58088      -58     
==========================================
+ Hits        41814    41880      +66     
+ Misses      16332    16208     -124     
 🚀 New features to boost your workflow:
 | 
| Artifact Size Benchmark Reportaarch64-alpine-linux-musl
 aarch64-unknown-linux-gnu
 libdatadog-x64-windows
 libdatadog-x86-windows
 x86_64-alpine-linux-musl
 x86_64-unknown-linux-gnu
 | 
| /merge | 
| View all feedbacks in Devflow UI. 
 
 The expected merge time in  
 | 
What does this PR do?
Fixes
stop_workerby dropping reference to the IoStack from the info fetcher.Motivation
What inspired you to submit this pull request?
Additional Notes
Anything else we should know when reviewing?
How to test the change?
Describe here in detail how the change can be validated.